Closed
Bug 680692
Opened 13 years ago
Closed 13 years ago
GPOS contextual chaining positioning does not work
Categories
(Core :: Graphics, defect)
Tracking
()
VERIFIED
FIXED
mozilla10
People
(Reporter: khaled, Assigned: khaled)
References
Details
(Whiteboard: [qa!])
Attachments
(4 files)
2.62 KB,
patch
|
jfkthame
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta-
|
Details | Diff | Splinter Review |
187.12 KB,
text/html
|
Details | |
7.92 KB,
image/png
|
Details | |
7.52 KB,
image/png
|
Details |
Due to a bug in HarfBuzz, contextual chaining positioning are broken and do not get applied it all. It have been fixed in HarfBuzz master (http://cgit.freedesktop.org/harfbuzz/commit/?id=cc1a8a938b4c13e76b58825a9e1951c4134e634a). The attached patch is the original upstream commit applied to mozilla-centeral, I built and tested it and it seem to work as expected.
Assignee | ||
Comment 1•13 years ago
|
||
This is a test file showing the issue with embedded woff font (the font is OFL licensed, base64 encoded just for convenience).
Assignee | ||
Updated•13 years ago
|
Attachment #554658 -
Attachment mime type: text/plain → text/html
Assignee | ||
Comment 2•13 years ago
|
||
Assignee | ||
Comment 3•13 years ago
|
||
Comment 4•13 years ago
|
||
I'd really prefer to take a full harfbuzz update soon, but if we don't get that done in good time before the next Aurora train leaves, we should consider cherry-picking this fix.
Status: UNCONFIRMED → NEW
Component: Layout: Text → Graphics
Ever confirmed: true
QA Contact: layout.fonts-and-text → thebes
Comment 5•13 years ago
|
||
Comment on attachment 554656 [details] [diff] [review] Backported upstream patch As bug 695857 (harfbuzz update) is still awaiting review, I think we should take this fix now as a stop-gap. Hence r=me.
Attachment #554656 -
Flags: review+
Comment 6•13 years ago
|
||
Pushed to mozilla-inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/0355183e7170
Target Milestone: --- → mozilla10
Do we need this in Firefox 9?
Comment 8•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0355183e7170
Assignee: nobody → khaledhosny
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 9•13 years ago
|
||
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #7) > Do we need this in Firefox 9? Well... we've been shipping the buggy behavior for several versions now, but it would be nice to fix it sooner rather than later. In particular, I believe (based on email correspondence with the font developer concerned) that this affects a new Arabic webfont that the BBC Arabic site is about to deploy. So until we fix it, that's a pretty major site that will look bad in Firefox. There are presumably other affected fonts/sites as well, though I don't know how widespread the issue is.
Comment on attachment 554656 [details] [diff] [review] Backported upstream patch Review of attachment 554656 [details] [diff] [review]: ----------------------------------------------------------------- I think we should do this. This is a fairly simple patch that will fix rendering on a major site.
Attachment #554656 -
Flags: approval-mozilla-beta?
Attachment #554656 -
Flags: approval-mozilla-aurora?
status-firefox7:
--- → affected
status-firefox8:
--- → affected
status-firefox9:
--- → affected
tracking-firefox9:
--- → ?
Comment 11•13 years ago
|
||
Comment on attachment 554656 [details] [diff] [review] Backported upstream patch [Triage Comment] * Approving for aurora given the affect to BBC's rollout * Denying for beta since this wouldn't cause a re-roll Please make sure to land this today to make the cutover.
Attachment #554656 -
Flags: approval-mozilla-beta?
Attachment #554656 -
Flags: approval-mozilla-beta-
Attachment #554656 -
Flags: approval-mozilla-aurora?
Attachment #554656 -
Flags: approval-mozilla-aurora+
Comment 13•13 years ago
|
||
The test file https://bug680692.bugzilla.mozilla.org/attachment.cgi?id=554658 is displayed like in the attachment https://bug680692.bugzilla.mozilla.org/attachment.cgi?id=554660 on: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0) Gecko/20100101 Firefox/9.0 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111121 Firefox/11.0a1 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a2) Gecko/20111121 Firefox/10.0a2 Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0 Mozilla/5.0 (Windows NT 6.1; rv:11.0a1) Gecko/20111121 Firefox/11.0a1 Mozilla/5.0 (Windows NT 6.1; rv:10.0a2) Gecko/20111121 Firefox/10.0a2 Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20100101 Firefox/9.0 Mozilla/5.0 (Windows NT 5.1; rv:11.0a1) Gecko/20111121 Firefox/11.0a1 Mozilla/5.0 (Windows NT 5.1; rv:10.0a2) Gecko/20111121 Firefox/10.0a2 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0) Gecko/20100101 Firefox/9.0 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0a2) Gecko/20111121 Firefox/10.0a2 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:11.0a1) Gecko/20111121 Firefox/11.0a1 Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20100101 Firefox/9.0 Mozilla/5.0 (X11; Linux x86_64; rv:10.0a2) Gecko/20111121 Firefox/10.0a2 Mozilla/5.0 (X11; Linux x86_64; rv:11.0a1) Gecko/20111122 Firefox/11.0a1 Is that correct ? Can I change the resolution to verified fixed ?
Comment 14•13 years ago
|
||
I believe that is correct. Khaled, please reopen this bug if it is not working the way you intended.
Status: RESOLVED → VERIFIED
Updated•13 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•